Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.github/workflows/analysis-coverage.yml:
- Around line 416-417: Add the Circles checkout and enablement to the
tests-client-sqlite job so client-mode Teams tests run under Circles as in
tests-maria/tests-pgsql: ensure the workflow job named tests-client-sqlite
includes the same checkout step for the Circles app and the "Enable Circles"
step that runs the command php occ app:enable circles (same as the existing
Enable Circles step in the diff), mirroring the setup used in
tests-maria/tests-pgsql; also consider doing the same for tests-oci if you want
OCI client-mode coverage.

In `@nc_py_api/teams.py`:
- Around line 114-122: The user_type and level properties currently call
MemberType(self._raw_data.get("userType", 1)) and
MemberLevel(self._raw_data.get("level", 0)) which will raise ValueError if the
API returns unknown enums; update these accessors (user_type and level) to
defensively handle invalid values by catching ValueError (or using the
Enum._missing_ hook) and returning a safe fallback (e.g., MemberType.USER and
MemberLevel.NONE) while still using self._raw_data.get(...) to obtain the raw
value; ensure the change references the same symbols (user_type, level,
MemberType, MemberLevel, and _raw_data) so behavior is forward-compatible with
unexpected API values.
- Around line 138-143: The `@dataclasses.dataclass` decorator on class Circle is
unnecessary because Circle defines a custom __init__ and doesn't use dataclass
features; remove the decorator line above class Circle (and mirror the same
removal you applied to Member) so Circle is a plain class with its existing
__init__ and _raw_data attribute.
- Around line 87-92: The `@dataclasses.dataclass` decorator on class Member is
misleading because Member defines a custom __init__ (and __repr__), so remove
the decorator line from the class definition and keep the existing
__init__/_raw_data handling (or alternatively, if you prefer using dataclass
semantics, replace the custom __init__ and __repr__ with dataclass field
declarations such as raw_data: dict and let dataclass generate
__init__/__repr__); in short, either delete the `@dataclasses.dataclass` decorator
from Member or refactor Member to declare dataclass fields and remove the manual
__init__/__repr__ so the decorator is used correctly.

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@tests/actual_tests/teams_test.py`:
- Line 121: The assertion currently hardcodes details.owner.user_id == "admin";
replace the hardcoded string with the test's actual acting-user id variable so
the check is environment-agnostic (e.g., assert details.owner.user_id ==
acting_user.user_id or assert details.owner.user_id == current_user.user_id —
locate the variable used in this test that represents the requester/test user
and use that instead of "admin").
- Around line 243-260: AsyncNextcloud instances (anc_user and anc_any) leak HTTP
connection pools because AsyncNextcloud provides no public close API; update the
test to explicitly close the underlying session adapters after using
AsyncNextcloud: wrap the operations that call anc_user.teams.join /
anc_user.teams.leave and anc_any.teams.get_members in try/finally blocks and in
the finally await closing of anc_user._session.adapter and
anc_user._session.adapter_dav (and do the same for anc_any) to ensure adapters
are closed and connections cleaned up.